Skip to content

SERVER-118939 Add support for mozJS build compatibility for 8.0 backport#5

Merged
aditya8061 merged 3 commits into
spidermonkey-esr140.7-cpp-onlyfrom
aditya.deshpande/SERVER-118939
Feb 5, 2026
Merged

SERVER-118939 Add support for mozJS build compatibility for 8.0 backport#5
aditya8061 merged 3 commits into
spidermonkey-esr140.7-cpp-onlyfrom
aditya.deshpande/SERVER-118939

Conversation

@aditya8061
Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Feb 4, 2026

(Automated Close) Please do not file pull requests here, see https://firefox-source-docs.mozilla.org/contributing/how_to_submit_a_patch.html

@github-actions github-actions Bot closed this Feb 4, 2026
@github-actions github-actions Bot locked and limited conversation to collaborators Feb 4, 2026
@aveselova aveselova reopened this Feb 5, 2026
Copy link
Copy Markdown
Collaborator

@aveselova aveselova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would suggest:

Then we can review and merge this via this PR rather than in slack

@aditya8061 aditya8061 requested a review from aveselova February 5, 2026 17:24
Copy link
Copy Markdown
Collaborator

@aveselova aveselova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good

Can you change the PR title to something that will make the commit message clearer, ex "Support older fmt version, delete github action, document explicit import"

@aveselova
Copy link
Copy Markdown
Collaborator

Also let's make sure evergreen looks good on master and 8.0 before we merge this

@aditya8061 aditya8061 merged commit 43ad77e into spidermonkey-esr140.7-cpp-only Feb 5, 2026
@aditya8061
Copy link
Copy Markdown
Collaborator Author

aditya8061 commented Feb 5, 2026

Yes - I will trigger evergreen patch on master. Plan is to first merge it to master and then backport these changes.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants